Skip to content

Support alternate gating methods#1215

Merged
jb3 merged 9 commits into
masterfrom
joseph/verification
Oct 5, 2020
Merged

Support alternate gating methods#1215
jb3 merged 9 commits into
masterfrom
joseph/verification

Conversation

@jb3
Copy link
Copy Markdown
Member

@jb3 jb3 commented Oct 5, 2020

This PR extends the verification cog to support users joining through alternate gating methods.

It will send a separate DM for any user verifying through a gating method which is not !accept and will completely disable the welcome DM.

image

@jb3 jb3 requested a review from a team as a code owner October 5, 2020 23:21
@jb3 jb3 requested review from MrHemlock and fiskenslakt and removed request for a team October 5, 2020 23:21
@ghost ghost added the needs 2 approvals label Oct 5, 2020
Comment thread bot/exts/moderation/verification.py Outdated
Comment thread bot/exts/moderation/verification.py Outdated
@ghost ghost added s: waiting for author Waiting for author to address a review or respond to a comment and removed needs 2 approvals labels Oct 5, 2020
@MarkKoz MarkKoz added a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 1 - high High Priority t: feature New feature or request labels Oct 5, 2020
@ghost ghost added needs 2 approvals and removed s: waiting for author Waiting for author to address a review or respond to a comment labels Oct 5, 2020
Copy link
Copy Markdown
Contributor

@lemonsaurus lemonsaurus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great overall. Couple nitpicks, but nothing that needs to block merge. fix if you want.

Comment thread bot/exts/moderation/verification.py Outdated
Comment thread bot/exts/moderation/verification.py
Comment thread bot/exts/moderation/verification.py Outdated
Comment thread bot/exts/moderation/verification.py Outdated
Comment thread bot/exts/moderation/verification.py Outdated
Comment thread bot/exts/moderation/verification.py Outdated
@ghost ghost added s: waiting for author Waiting for author to address a review or respond to a comment and removed needs 1 approval labels Oct 5, 2020
@ghost ghost added needs 1 approval and removed s: waiting for author Waiting for author to address a review or respond to a comment labels Oct 5, 2020
@ghost ghost removed the needs 1 approval label Oct 5, 2020
@jb3 jb3 merged commit 3ac7bba into master Oct 5, 2020
@jb3 jb3 deleted the joseph/verification branch October 5, 2020 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 1 - high High Priority t: feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants